Skip to content

Dots in database name are no longer deprecated. #2452

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 7, 2025

Conversation

Lojjs
Copy link
Contributor

@Lojjs Lojjs commented Jun 27, 2025

Backporting relevant parts of #2443 to 5.26 as the previous deprecation has been removed from 5.26.9

Comment on lines -22 to -23
Database names are the only identifier for which dots do not need to be quoted.
For example `main.db` is a valid database name.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wouldn't these two make sense to keep? 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is the sentence which was also removed from the 2025.x docs, so I followed the pattern here.

@Lojjs Lojjs force-pushed the 5.x-removal-deprecation branch from b9119fa to 17cf86e Compare July 7, 2025 07:49
@Lojjs Lojjs force-pushed the 5.x-removal-deprecation branch from 17cf86e to c9ea71e Compare July 7, 2025 07:51
@neo4j-docops-agent
Copy link
Collaborator

neo4j-docops-agent commented Jul 7, 2025

Thanks for the documentation updates.

The preview documentation has now been torn down - reopening this PR will republish it.

Copy link
Collaborator

@renetapopova renetapopova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@Lojjs Lojjs merged commit e2e4985 into neo4j:5.x Jul 7, 2025
4 checks passed
@Lojjs Lojjs deleted the 5.x-removal-deprecation branch July 7, 2025 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants